Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat: husky 환경 설정 및 github autoassign 등록 #8

Merged
merged 8 commits into from
Feb 2, 2024
Merged

Feat: husky 환경 설정 및 github autoassign 등록 #8

merged 8 commits into from
Feb 2, 2024

Conversation

Nahyun-Kang
Copy link
Contributor

@Nahyun-Kang Nahyun-Kang commented Feb 1, 2024 •
edited
Loading

개요

  • 개요는 변경 사항 및 관련 이슈에 대해 간단하게 작성해주세요.
  • 해당 PR에 대한 리뷰어와 라벨을 생성해 주세요.

관련 이슈

closed: #9


작업 사항

  • husky 설치 및 세팅을 진행했습니다.
  • lint-staged와 commitlint 설치 및 세팅도 진행했습니다.
  • github autoAssign 기능을 추가했습니다.
  • 허스키 실행 후, 발견한 이슈들을 해결했습니다.

참고 사항 (optional)

제가 한 허스키 설정 관련하여 노션에 정리해두었습니당!
🔗husky 설정 관련 Notion

  • eslint-config-prettier가 없어서 husky lint가 작동이 안 되는 문제가 있었습니다
  • 해결 방법 : yarn --dev eslint-config-prettier 를 설치하고 해결되었습니다!
  • 해당 라이브러리를 선택한 이유 : git hook을 손쉽게 제어하도록 도와주기 때문...!

스크린샷


커밋메시지 컨벤션 어길 시

허스키 커밋린트 설정

리뷰어에게

  • 일단 .husky >-> gitignore 파일에 *를 없애두어서 pr에 반영될 수 있도록 설정해두었는데 pull 받으시고 추후에 gitignore 파일에 * 한 글자 넣어주시면 될 것 같습니당!

@Nahyun-Kang Nahyun-Kang added the Feat 구현 label Feb 1, 2024
@Nahyun-Kang Nahyun-Kang self-assigned this Feb 1, 2024
Eugene-A-01
Eugene-A-01 reviewed Feb 1, 2024
View reviewed changes
commitlint.config.js
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주석 달아주셔서 뭐가 어디에 영향미치는지 바로 알겠네요!!
나현님 감사합니다!
LGTM👍

@seoyoung-min seoyoung-min changed the title Feature/husky Feat: husky Feb 1, 2024
@seoyoung-min seoyoung-min changed the title Feat: husky Feat: husky 환경 설정 Feb 1, 2024
seoyoung-min
seoyoung-min approved these changes Feb 1, 2024
View reviewed changes
Copy link
Contributor

@seoyoung-min seoyoung-min left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

노션에 정리까지 해주시다니 정말 감사합니다ㅠㅠ!
완전 이해가 쏙쏙 잘되네요✨ LGTM👍

@Nahyun-Kang Nahyun-Kang changed the title Feat: husky 환경 설정 Feat: husky 환경 설정 및 github autoassign 등록 Feb 1, 2024
ParkSohyunee
ParkSohyunee approved these changes Feb 1, 2024
View reviewed changes
Copy link
Contributor

@ParkSohyunee ParkSohyunee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

나현님, 허스키 설정하시느라 고생많으셨습니다!
노션 읽으면서 나현님께서 허스키 설정하시면서 재밌다고 느낀 포인트를 조금 알 것 같다는 생각도 드네요 ㅎㅎ 덕분에 이해가 너무 잘되었습니다!! 감사합니다.👍

kanglocal
kanglocal approved these changes Feb 2, 2024
View reviewed changes
Copy link
Contributor

@kanglocal kanglocal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

나현님 최고 👍 👍 👍 👍
너무 고생 많으셨습니다!!!

@Nahyun-Kang Nahyun-Kang merged commit e7e1417 into 8-Sprinters:dev Feb 2, 2024
1 check passed
@Nahyun-Kang Nahyun-Kang deleted the feature/husky branch February 21, 2024 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seoyoung-min seoyoung-min seoyoung-min approved these changes

@ParkSohyunee ParkSohyunee ParkSohyunee approved these changes

@kanglocal kanglocal kanglocal approved these changes

@Eugene-A-01 Eugene-A-01 Eugene-A-01 approved these changes

Assignees

@Nahyun-Kang Nahyun-Kang

Labels
Feat 구현
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

허스키 설정 이후, prettier / eslint 오류 발생
5 participants
@Nahyun-Kang @Eugene-A-01 @kanglocal @ParkSohyunee @seoyoung-min